Update deck.rs #2

TheDeveloper101 · 2022-05-24T19:42:19Z

Created a rough method to be able to get a card. But the method requires the name of the card to be passed in as an AbstractCard. Also the method returns an AbstractCard and it would be nice if it could return a string. I am not sure how to make it such that the method will simply return a card from the HashBag but this is what I could come up with

TylerBloom · 2022-05-24T21:51:58Z

src/model/deck.rs

@@ -59,6 +61,10 @@ impl Deck {
        self.mainboard.insert_many(card, count)
    }

+    pub fn get_card(&self, card: &AbstractCard) -> AbstractCard {


I doubt the usefulness of this method. Let's say the deck does contain the given card in the mainboard. You are getting it, cloning it, and returning it to the user. However, the returned card should contain all the same information that the given card does. Based on our conversation elsewhere, I believe you want to add methods on the AbstractCard struct.

Update deck.rs

1aa519d

TylerBloom reviewed May 24, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update deck.rs #2

Update deck.rs #2

TheDeveloper101 commented May 24, 2022

TylerBloom May 24, 2022

Update deck.rs #2

Are you sure you want to change the base?

Update deck.rs #2

Conversation

TheDeveloper101 commented May 24, 2022

TylerBloom May 24, 2022

Choose a reason for hiding this comment